Skip to content

Update manifest schema to v1.25 across all templates #15217

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Copilot wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

Update manifest schema to v1.25 across all templates #15217

Copilot wants to merge 4 commits into from

Conversation

Copy link
Contributor

Copilot AI commented Jan 26, 2026 •
edited
Loading

Updates Microsoft Teams App Manifest from v1.24/v1.23 to v1.25 per microsoft/json-schemas@f323b42.

Schema Change

v1.25 adds default value for composeExtensions.composeExtensionType:

"composeExtensionType": {
  "type": "string",
  "enum": ["botBased", "apiBased"],
  "default": "botBased"  // new in v1.25
}

Non-breaking: explicit "apiBased" values remain unchanged.

Changes

Template files (121 files)

  • Updated $schema URL: v1.24 → v1.25
  • Updated manifestVersion: "1.24" → "1.25"
  • Covers all template categories: vs/csharp, vsc/{js,python,ts}, vsc/common, unused/*

Manifest package

  • Added src/json-schemas/teams/v1.25/MicrosoftTeams.schema.json
  • Generated TeamsManifestV1D25 types via quicktype
  • Updated TeamsManifestLatest alias: V1D21 → V1D25
  • Added v1.25 converter mappings in union type and converter map
Original prompt

This section details on the original issue you should resolve

<issue_title>Update App Manifest Version to v1.25 in each template</issue_title>
<issue_description># App Manifest Schema Update v1.25

Latest Schema Commit: f323b42

🎯 Impacted Properties We Support

The following properties used in our templates have been updated and require attention:

composeExtensions.composeExtensionType ✅ NON-BREAKING

Change: Added a default value of 'botBased'.

Impact: If the property is omitted in new manifests, it will default to 'botBased'. This could affect integrations or logic that rely on the explicit presence or value of this property, potentially altering extension behavior if the default is not intended.

📋 Action Required

Please update the following files in each template repository:

  • manifest.json.tpl
  • manifest.local.json.tpl (if applicable)
  • manifest.dev.json.tpl (if applicable)

Note: Ensure all impacted properties listed above are properly updated according to the new schema requirements.

🔗 Related Work Item

Azure DevOps Work Item

This issue was automatically generated by the Dependency Manager.</issue_description>

Comments on the Issue (you are @copilot in this section)

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits January 26, 2026 23:03
@tecton
Update manifest version to v1.25 in all 118 template files
4cced00 
Co-authored-by: tecton <886116+tecton@users.noreply.github.com>
@tecton
Add v1.25 schema and update manifest package types
23ab491 
Co-authored-by: tecton <886116+tecton@users.noreply.github.com>
@tecton
Update remaining 3 v1.23 manifest files to v1.25
04ce3b3 
Co-authored-by: tecton <886116+tecton@users.noreply.github.com>
Copilot AI changed the title [WIP] Update app manifest version to v1.25 in templates Update manifest schema to v1.25 across all templates Jan 26, 2026
Copilot AI requested a review from tecton January 26, 2026 23:10
@tecton tecton marked this pull request as ready for review January 29, 2026 08:59
@codecov
Copy link

codecov bot commented Jan 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.24%. Comparing base (8cf9cf9) to head (04ce3b3).
⚠️ Report is 29 commits behind head on dev.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##              dev   #15217    +/-   ##
========================================
  Coverage   89.24%   89.24%            
========================================
  Files         618      618            
  Lines       38004    38008     +4     
  Branches     7772     7554   -218     
========================================
+ Hits        33917    33921     +4     
  Misses       1972     1972            
  Partials     2115     2115

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tecton tecton Awaiting requested review from tecton tecton is a code owner

@HuihuiWu-Microsoft HuihuiWu-Microsoft Awaiting requested review from HuihuiWu-Microsoft HuihuiWu-Microsoft is a code owner

@wh-alice wh-alice Awaiting requested review from wh-alice wh-alice is a code owner

@qinzhouxu qinzhouxu Awaiting requested review from qinzhouxu qinzhouxu is a code owner

@llhmm01 llhmm01 Awaiting requested review from llhmm01 llhmm01 is a code owner

@Sherylueen Sherylueen Awaiting requested review from Sherylueen Sherylueen is a code owner

@Alive-Fish Alive-Fish Awaiting requested review from Alive-Fish Alive-Fish is a code owner

@yuqizhou77 yuqizhou77 Awaiting requested review from yuqizhou77 yuqizhou77 is a code owner

@qinqingxu qinqingxu Awaiting requested review from qinqingxu qinqingxu is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@tecton tecton

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update App Manifest Version to v1.25 in each template

2 participants

@tecton